Skip to content

burn(bug): Fix unused retSlot variable in LIR lowering#44

Draft
bkmashiro wants to merge 1 commit intomainfrom
burn/bug/9r14tp/fix-unused-retslot-variable-in-lir-lower
Draft

burn(bug): Fix unused retSlot variable in LIR lowering#44
bkmashiro wants to merge 1 commit intomainfrom
burn/bug/9r14tp/fix-unused-retslot-variable-in-lir-lower

Conversation

@bkmashiro
Copy link
Copy Markdown
Owner

Task

  • ID: 01KNHZFNWBVC163XXGNY9R14TP
  • Type: bug
  • Priority: 3
  • CLI: claude

Description

In src/lir/lower.ts around lines 638-642, the variable retSlot is declared but never used — the return_value instruction is emitted with srcSlot directly instead. Investigate whether retSlot should be used (i.e., the instruction should copy srcSlot into the $ret scoreboard slot explicitly) or whether the variable is a dead refactoring artifact and should be removed. Read the surrounding context and the emit stage handling of return_value instructions to determine the correct fix, then apply it.

Changes

src/__tests__/lir/lower.test.ts | 32 ++++++++++++++++++++++++++++++++
 src/lir/lower.ts                |  1 -
 2 files changed, 32 insertions(+), 1 deletion(-)

Generated by burn-harness

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant